Skip to content

Conversation

@MIchaelMainer
Copy link
Collaborator

@MIchaelMainer MIchaelMainer commented Aug 14, 2025

Installation id is not necessary

Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings August 14, 2025 21:10
@MIchaelMainer MIchaelMainer requested a review from a team as a code owner August 14, 2025 21:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates parameter names in the GitHub Actions workflow to use the correct format with hyphens instead of underscores for the actions/create-github-app-token@v2 action, and removes the unnecessary installation_id parameter.

  • Changed parameter names from underscore format to hyphen format (app_idapp-id, private_keyprivate-key)
  • Removed the installation_id parameter which is not required for this action

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

private_key: ${{ secrets.RELEASE_PLEASE_TOKEN_PROVIDER_PEM }}
installation_id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_INSTALLATION_ID }}

app-id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_APP_ID }}
Copy link

Copilot AI Aug 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parameter name should be app_id with underscores, not app-id with hyphens. According to the actions/create-github-app-token@v2 documentation, the correct parameter name is app_id.

Suggested change
app-id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_APP_ID }}
app_id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_APP_ID }}

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot, you are hallucinating. Please review the documentation again.

https://github.com/actions/create-github-app-token

installation_id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_INSTALLATION_ID }}

app-id: ${{ vars.RELEASE_PLEASE_TOKEN_PROVIDER_APP_ID }}
private-key: ${{ secrets.RELEASE_PLEASE_TOKEN_PROVIDER_PEM }}
Copy link

Copilot AI Aug 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parameter name should be private_key with underscores, not private-key with hyphens. According to the actions/create-github-app-token@v2 documentation, the correct parameter name is private_key.

Suggested change
private-key: ${{ secrets.RELEASE_PLEASE_TOKEN_PROVIDER_PEM }}
private_key: ${{ secrets.RELEASE_PLEASE_TOKEN_PROVIDER_PEM }}

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot, you are hallucinating. Please review the documentation again.

https://github.com/actions/create-github-app-token

@MIchaelMainer MIchaelMainer merged commit 4192a5f into main Aug 14, 2025
5 checks passed
@MIchaelMainer MIchaelMainer deleted the mmainer/rp_param_names branch August 14, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants